Update igc.cc with new Header type S (#446)

Add the new Header type S to allowed values on line 308

hide magic time value from test mode decisions. (#443)

Fix test generating gpx schema violations. (#444)

TODO: Detect bad input coordinates.

use QDebug in logging. (#438)

* use QDebug in logging.

This gives us full access to all the methods, stream operators and
manipulators that QDebug has access to.

* silence warning on unhandled fix values in nmea.

Fix potential slicing with Filter class. (#440)

detected by clazy as:
warning: Polymorphic class Filter is copyable. Potential slicing. [-Wclazy-copyable-polymorphic]

fix NMEA date fiddling. (#437)

Three errors were corrected:
1) SECONDS_PER_DAY is assumed to be an integer type.  It was recently
errorously changed to to double.
2) when backfilling dates we need to subtract a day at rolloever,
not addit.
3) If we have a date, then it came with a time.  Don't set the backfill
time to 23:59:59.

Add test coverage for the above.

Delete file erroneously added in #409. (#435)

Merge pull request #409 from gpsbabel/no_new_string

Remove remnants of NEW_STRINGS

Merge pull request #433 from gpsbabel/cleanups_201912

Cleanups: make a gbfreadbuf, move units from defs and into dedicated file, reduce xstrdup,xmalloc use, reduce CString/QString roundtrips, more.

tweaks for format vectors. (#434)

flush out hiketech capability array.

use copy constructor of ff_vecs_t when creating unified vector list.

Cleanup train keeps rollin' along
defs.h: use c++ true and false. Replace macros with constexprs.
gbser_win.cc don't use Windows style TRUE/FALSE.
kml.cc: integration suggestions.
msvc build: use units.h

Add util.h to various build files.

Mechanically move protos and structs for units.cc from defs.h to units.h

A bunch of busy-work in coce that approximarly never is actually run but solely for the hope of hushing clang, clion, and codacy.

Fix ordering in optional altitude test in KML reader

Warning tweaks in defs for safety in macro expansion.

Largely automated cleanups in NMEA.

Rework KML linestring coordinate reader. About a 21x speedup on a file
with 38,000 coordinates tuples.

Make WAYPT_SET act more like a function after macro expansion.

Lowranceusr: internal tightening of types, better scope management.

Make GPX internal promotions/reductions more specific.
Automated cleanups courtesy Clion.

Add gbfreadbuf to simplify object storage.

Make tpo less weird. Rely on xmalloc/xfree way less. Reduce internal copies.

In Teletype, seek over unknown instead of allocating, reading, and tossing.

use nullptr instead of empty functions (#432)

for function pointers passed to route_disp_all, track_disp_all
and ff_vecs_t exit.

clean up filter includes. (#431)

remove obsolete definition of filter_vecs_t.
don't included filterdefs.h in each filters .cc file.
add QVector include to each filters .h file.

use QVectors for filter and format arguments. (#430)

* use QVectors for filter and format arguments.

This includes a work around MSVC 2015 errors C2440, C2664.  See
the conditional code in defs.h.

* restore accidental deletion of include file.

standardize arg proccessing loop structure and naming. (#429)

* standardize arg processing loops in vec handling.

* standardize var naming in arg processing loops.

using QStrings in filter and format vecs. (#428)

This includes:
our format list entry vecs_t,
out filter list entry fl_vecs_t and
our format interface ff_vecs_t.

Fix validation of options supplied with a format or filter.
Previously unknown options were only detected if all given options
were unknown.  Now any unknown option will be reported, irregardless
of any other options supplied.
The behavoir of ignoring unknown options is unchanged, they are ignored
except for the warning.

improve format/filter vector validation. (#427)

Merge pull request #426 from tsteven4/using

clang-tidy modernize-use-using

delete some dangling structure-tags.

clang-tidy modernize-use-using.

replace typedef syntax.
this was done by hand.

Merge pull request #424 from tsteven4/vec_containers

use Qt containers for format/style/filter vecs.

use Qt containers for format/style/filter vecs.

Merge pull request #423 from tsteven4/clazy-fixes

* fix c++11 range-loop might detach Qt container

clazy 1.6, clazy-range-loop

* fix Missing reference on non-trivial type

clazy 1.6, clazy-function-args-by-ref

fix Missing reference on non-trivial type

clazy 1.6, clazy-function-args-by-ref

fix c++11 range-loop might detach Qt container

found with clazy 1.6

fix format vector consistency issues (#422)

* fix format vector consistency issues.

* fix leaks in sort_and_unify_vecs.

* fix vec init for msvc2015.

* Revert "fix vec init for msvc2015."

This reverts commit 02b85801ee4d1de24101713fbd85c33238da6183.

* Revert "fix leaks in sort_and_unify_vecs."

This reverts commit d634c21f9f7767d198750e61ba426eb7850127a9.

* only validate formats for real types.

Replace libusb 0.1 usage with libusb 1.0 (#396)

Replace our modified libusb 0.1.12 with libusb 1.0.22 on macos. On macos we statically link to our inclcuded libusb.
Use libusb 1.0 instead of libusb 0.1 on linux. On linux we dynamically link to a distribution provided libusb, or optionally build without libusb support.

update Qt to 5.12.6 (#420)

Correct Bundle Identifier for macos using recommended reverse (#419)

DNS format.

Build a bundle with the cmake flow on macos.

Delete gpsbabelfe script that messed with LD_LIBRARY_PATH and
then executed gpsbabelfe-bin. This file was obsolete, and
it's existence upset Xcode (entitlement file handling).

clean up "No newline at end of file" (#418)

and delete an empty test file.

Merge pull request #410 from gpsbabel/docbook2

Dev Doc fix: Document Qt, Qmake better.

Merge pull request #411 from gpsbabel/docbook3

Repair inconsistencies in xsl docbook source

Merge pull request #413 from gpsbabel/robertlipe-patch-1

Notes shouldn't be part of this CL

Merge pull request #414 from gpsbabel/robertlipe-patch-2

Delete miniHomer-gps-utc-offset.xml

Merge pull request #415 from gpsbabel/robertlipe-patch-3

Delete miniHomer-gps-utc-offset.xml

Merge pull request #416 from gpsbabel/robertlipe-patch-3-1

Delete miniHomer-gps-week-rollover.xml

Merge pull request #417 from gpsbabel/docbook5

Docbook cleanup

Merge pull request #395 from gromit1811/fit_write_support

Implement FIT writing support

m241: Document serial port hardware

Hint that yellow m241 uses SiLabs CP210x. Red m241 is dead to us and the whole company is even more dead.

Delete miniHomer-gps-week-rollover.xml

Delete miniHomer-gps-utc-offset.xml

Delete notes

Delete miniHomer-gps-utc-offset.xml

Shouldn't be part of 409 commit

I failed a wrestling match with Github

Notes shouldn't be part of this CL

I failed a wrestling match with Github

Cleanup of dangling doc files for dead formats

Formats in deprecated/  don't need live doc.
Reformatted our largest xmldoc files.

Repair inconsistencies in xsl docbook source

Dev Doc fix: Document Qt, Qmake better.

Better enumerate dependencies
Describe Qt Crerator
Improve internal linking
Fix trailing spaces

Dev Doc fix: Document Qt, Qmake better.

Better enumerate dependencies
Describe Qt Crerator
Improve internal linking
Fix trailing spaces

Remove remnants of NEW_STRINGS

NEW_STRINGS haven't been new for many years at this point. I'm tired of looking at that code. Most of what's left is in paths never executed or tested anyway. :-/

Implement FIT writing support

Generate FIT courses from tracks+waypoints. Waypoints get mapped to course
points (generic by default, left/right turns if waypoint names contain the
string "left"/"right" or their equivalents in German, French, Spanish,
Italian).

If the original track contains neither time stamps nor speed values,
a fixed speed of 10 km/h gets assigned to the whole course.

Resulting FIT files have been tested and are accepted by a Garmin Edge 130
unit, but are assumed to be usable with any GPS unit supporting FIT.

Track segement markers are ignored at the moment.

update qt to 5.12.5 (#403)

Merge pull request #400 from jspricke/manpage_xml

Fix manpage xml

Merge pull request #401 from spacelama/master

Don't dump garmin_fit from Lezyne "Super Fit" devices just because occasionally the endian field is not 0 or 1, leading to the question if that's an endianness field at all.

correct reference file for garmin_fit lezyne super gps - probably originally converted using wrong flags

The lezyne "super gps", every 10th ride or so, outputs a .fit file
that is entirely valid other than a weird non 0/1 endian field.  Don't
dump the data just because of that.

Fix manpage xml

fix lowranceusr4 writer issues (#376)

* fixes for lowraneusr writer for versions >= 4.

use fs_chain_find when looking for FS_LOWRANCEUSR4 data.
and check for nullptr to verify FS_LOWRANCEUSR4 data exists.

* using rounding in lowranceusr lat/lon conversions.

This gives as a chance of usr->gpx->usr matching.

Update gpsbabel generated usr reference files for these
rounding differences.

* fix lowranceusr4 writer issues.

when writing embedded UTF-16LE strings in lowranceusr version >=4
don't inlucde BOM.

don't offset returned values from lowranceusr4_jd_from_timestamp
by 12 hours, thus matching the inverse transformation in
lowranceusr4_get_timestamp.

when writing lowranceusr version >=4 trails output the given number
of mysterious attribute bytes, i.e. 0.

This almost enables a gpx -> usr -> gpx test loop, which is included
but commented in lowranceusr.test.

* fixes to enable lowranceusr v4 round trip testing.

use templates for lowranceusr*_find_desc_from_icon_number
and lowranceusr*_find_icon_number_from_desc.

enhance lowranceusr*_find_icon_number_from_desc to
recongnize icon-n where n is an integer as icon number n.
This is necessary for the v4 round trip test.

declare lowranceusr*_find_desc_from_icon_number to be static
to limit visibilty. Drop const return type.

declare lowranceusr4_find_color_from_icon_number_plus_color_index
to be static to limit visiblity.

* fix new lowranceusr memory leak.

fix minor error in garmin_fit lat/lon conversion. (#398)

Standardize resource access (#393)

* standardize resource access.

avoid search paths (use :/).
specify extensions.
ok/cancel only on windows.

* fix missing icon for file input in options dialog.

It appears the image was renamed when used on the main window UI
in the inputFileNameBrowseBtn.
The same icon, open.png, is now used on the options dialog for
OPTinFile.
This fixes #392.

fix interaction between translation and lights. (#390)

When using a language other than English the initial painting of
the gui mainwindow showed the placeholder text for the labels next
to the waypoints, routes and tracks check boxes instead of the
lights.  This was issue #389.

don't compile unwanted module serial_unix.cc on mac. (#388)

* don't compile serial_unix.cc on mac.

* remove unnecessary conditional in serial_unix.cc

File is no longer in SOURCES for mac, so we don't need to use a
conditional to skip the contents on mac.

Update README.contrib to reference GitHub (#383)

Merge branch 'master' of https://github.com/gpsbabel/gpsbabel

Fix a missed newline after Verion in debug code for Garmin Protocol.

Merge pull request #377 from jspricke/no_makeflags

Don't pass MAKEFLAGS to $(MAKE)

Fix Garmin debugging for "funny" characters. Isalnum() prints single byte characters that aren't ASCII, making funny business in iTerm. Yes, all those should be refactored into a single "print buffer".

Try to make the 'resettime' work per
https://github.com/gpsbabel/gpsbabel/issues/350
Debugging shows it's sending a time to my USB Garmin, but since this seems
to be a non-rollover device, the time is borked on mine anyway.

Don't pass MAKEFLAGS to $(MAKE)

make will pass them automatically. Also they are not meant to be
consumed by make as they are stripped:

Thus, if you do ‘make -ks’ then MAKEFLAGS gets the value ‘ks’.

https://www.gnu.org/software/make/manual/html_node/Options_002fRecursion.html

Merge pull request #374 from tsteven4/jeepswarn

Fix g++-8 warnings in jeeps.

Fix g++-8 warnings in jeeps.

modify position filter algorithm. (#373)

For routes and tracks, the algorithm is modified to
only remove consecutive points until the distance limit is
exceeded.  This prevents simulataneous removal of points on
an outgoing and return leg, which can lead to gross corruption
of the route/track shape.

A test case that demonstrates the problem with the previous
algorithm is added.

spelling fixes for GUI (#372)

* clion spelling fixes for GUI.

Care must be taken to not lose translations!

* fix clang tidy performance-unnecessary-value-param

Gdb string read (#364)

* harden gdb reader to long string input.

fix bug in the gdb reader that resulted in file position being lost
if a string was truncated when reading due to the finite buffer size.

fix bug in the gdb reader that could result in data being written outside
the provided buffer.

enhance gdb readers to remove length limit on reading strings.

* whitespace correction.

clion spell fixes, and a few cet retirements. (#370)

* clion spell fixes, and a few cet retirements.

cet retirements were unused.

* more spelling fixes.

resharper fixes (#368)

* fix "redundant statement", "redundant declaration"

* fix newline at eof.

roll Qt to 5.12.4. (#369)

remove dependency on glob. (#367)

* remove dependency on glob.

and have magproto invoke the rd_deinit function when it uses the
gpx reader (uggh).

* regenerate configure on linux.

fix potential memory leaks. (#365)

* fix potential memory leaks.

* fix more memory leaks.

A method to find these is to add -DQT_ASCII_CAST_WARNINGS and
then look for xstrdup in the warnings.

* a few cleanups clazy caught.

Track filter fix (#363)

* Fix compatibility for track filter GUI<->CLI.

Add a week unit for the track filter move option,
in addition to the existing day, hour, minute and second units.

Allow combination of units for track filter move option, e.g.
1w2d3h4m-5s = 1 week + 2 days + 3 hours + 4 minutes - 5 seconds.
This matches historical use, and is expected by the GUI.

Add the week unit to the GUI track filter ui.

Change the limits on the track filter move specs:
week: +/-10000 (unit didn't exit before)
day,hour,minute,second: +/- 100000 (+/-2000 for day before, +/-100 on others).

This resolves #357.
This resolves #358.

* add documentation for track filter move option.

fix package app booboo. (#366)

dmg location was wrong so our attempt to remove any existing
dmg did not work.

enable gui to load Qt provided meta catalogs. (#360)

QTranslate::load fails to load a meta catalog if
the absolute file name is given as the filename parameter, and
the directory parameter is omitted.
It succeeds if the file name is provided as the filename parameter
and the directory is provided as the diretory parameter.

This should resolve #359.

fix stringop-overflow= warning with g++-9. (#362)

Additionally, produce a warning for any conversion errors when
reading XT_TIMET_TIME_MS fields in an xcsv file.

Fix crash on download from Navin/ZNEX miniHomer (#352)

This adds the options gps-utc-offset and gps-week-rollover to the
miniHomer driver. By adding them solely to the skytraq driver they were
initialized with nullptr for the miniHomer ultimately leading to a
crash.

Signed-off-by: Konrad Gräfe <konradgraefe@aol.com>

update Free Software Foundation address. (#354)

* update Free Software Foundation address.

* revert strptime fsf address change.

fix format-overflow= warning with g++-8 in mkshort.cc (#361)

This is Fedora patch 0006-Fix-Wformat-overflow.patch.

correct name of Exec in desktop file. (#353)

The name of the Exec changed in
https://github.com/gpsbabel/gpsbabel/pull/310